Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#4: Fix naming conventions on containers #12

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

cwschilly
Copy link
Collaborator

@cwschilly cwschilly commented Jan 30, 2025

Fixes #4

Moves compiler version to immediately follow the compiler. For example:

  • ubuntu-clang14
  • ubuntu-gnu11-kokkos-cpu-openmp
  • fedora-clang17-mpi

Also updates pipelines to only push images on merges to main or develop. (Images will build in CI, but only push to the container registry when the PR is merged.)

@cwschilly cwschilly linked an issue Jan 30, 2025 that may be closed by this pull request
@cwschilly cwschilly changed the title #4: append version to compiler; only push images on main or develop #4: Fix naming conventions on containers Jan 30, 2025
@cwschilly cwschilly requested a review from fnrizzi January 30, 2025 18:37
@fnrizzi fnrizzi merged commit f4dd08c into main Jan 31, 2025
12 checks passed
@cwschilly cwschilly deleted the 4-fixrevise-naming-convention-for-all-containers branch January 31, 2025 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix/revise naming convention for all containers
2 participants